Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[1.0] Add the sass plugin to this repo #871

Closed
wants to merge 2 commits into from

Conversation

danielfarrell
Copy link
Contributor

As discussed in #845, this probably belongs in this repo so I'm contributing it.

@danielfarrell danielfarrell changed the title Add the sass plugin to this repo [1.0] Add the sass plugin to this repo Apr 28, 2017
@KyleAMathews
Copy link
Contributor

This looks great! Thanks for adding this. Could you add me as an owner on NPM when you have a chance as well?

Also, it looks like you planned to allow people to pass it options? It doesn't look like that was implemented yet?

@KyleAMathews
Copy link
Contributor

Oh haha NVM, just saw on NPM you already added me 👍

Can add this back once options get implemented
@danielfarrell
Copy link
Contributor Author

There should probably be some options eventually, but you are right that that isn't implemented yet. I just pulled that out of the README for now.

@KyleAMathews
Copy link
Contributor

Merged it locally! Thanks for the contribution! https://github.com/gatsbyjs/gatsby/tree/1.0/packages/gatsby-plugin-sass

@danielfarrell danielfarrell deleted the add-sass-plugin branch April 28, 2017 15:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants